The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
litellm	Ready	Preview, Comment	Feb 16, 2026 3:03am

All committers have signed the CLA.

Greptile Summary

This PR removes a redundant duplicate call to _common_add_spend_log_transaction_to_daily_transaction in the agent spend logging path. The original code called this helper twice with effectively identical payloads (payload_with_agent_id was {**payload, "agent_id": payload["agent_id"]}, which is identical to payload since agent_id is already present). This doubled the JSON metadata parsing and status computation work on every agent spend log.

• Removed the unnecessary payload_with_agent_id construction and the second _common_add_spend_log_transaction_to_daily_transaction call in add_spend_log_transaction_to_daily_agent_transaction
• The method now reuses base_daily_transaction from the first call, matching the pattern used by team, user, tag, and other variants
• Added a regression test verifying the common helper is called exactly once per request (mock-only, no network calls)
• Net deletion of 13 lines of redundant code with no behavioral change

Confidence Score: 5/5

• This PR is safe to merge — it removes provably redundant code with no behavioral change and includes a regression test.
• The change is minimal and clearly correct. The removed code was a redundant second call to the same helper with an identical payload. The fix aligns the agent method with the established pattern used by all other entity-type variants (team, user, tag, end_user). The regression test validates the fix. No new logic is introduced.
• No files require special attention.

Sequence Diagram

sequenceDiagram
    participant Caller as Spend Logger
    participant Agent as add_spend_log_transaction_to_daily_agent_transaction
    participant Common as _common_add_spend_log_transaction_to_daily_transaction
    participant Queue as daily_agent_spend_update_queue

    Note over Agent: Before Fix (2 calls)
    Caller->>Agent: payload (with agent_id)
    Agent->>Common: payload, "agent" (1st call)
    Common-->>Agent: base_daily_transaction
    Agent->>Agent: construct payload_with_agent_id (redundant copy)
    Agent->>Common: payload_with_agent_id, "agent" (2nd call - DUPLICATE)
    Common-->>Agent: base_daily_transaction (same result)
    Agent->>Queue: add_update(daily_transaction)

    Note over Agent: After Fix (1 call)
    Caller->>Agent: payload (with agent_id)
    Agent->>Common: payload, "agent" (single call)
    Common-->>Agent: base_daily_transaction
    Agent->>Queue: add_update(daily_transaction)

_{Last reviewed commit: 87ba840}

The 6 failing "LiteLLM Unit Tests (Matrix)" checks (integrations, llms, proxy-guardrails, proxy-unit-a, proxy-unit-b, root) are pre-existing failures that affect all fork PRs due to missing repo secrets. These same tests fail on unrelated docs-only PRs (e.g., #21283, #21282).

All checks that can run on fork PRs pass successfully.

Fixed with #21187